Conversation
… code - Add nullable annotations to ITailBlazorTableModel interface properties (RowClass, ChildRowClass) - Remove redundant using statements (now covered by ImplicitUsings) - Remove redundant null check in OnParametersSet method These changes improve null safety and leverage .NET 9 features more effectively. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <table class="@Class"> | ||
| <CascadingValue Value="this"> | ||
| @ChildContent |
There was a problem hiding this comment.
The CascadingValue now sets Value="this", which passes the literal string "this" rather than the current TailBlazorTable instance. Any child components that consume the cascading parameter (e.g., [CascadingParameter] TailBlazorTable table) will no longer receive the expected object and will either get a string or null, leading to runtime casting errors and broken access to table state. This should remain Value=this so the component instance is cascaded.
Useful? React with 👍 / 👎.
2 participants
No description provided.